add clone back into menu #15240
add clone back into menu #15240
Conversation
datahub-cyborg
bot
added
the
needs-review
Bundle ReportChanges will increase total bundle size by 598 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: datahub-react-web-esmAssets Changed:
Files in
|
|
✅ Meticulous spotted 0 visual differences across 1015 screens tested: view results. Meticulous evaluated ~9 hours of user flows against your PR. Expected differences? Click here. Last updated for commit d99ccc8. This comment will update as new commits are pushed. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
chriscollins3456
left a comment
chriscollins3456
left a comment
There was a problem hiding this comment.
overall looks good! suggestion on updating the name here to have (copy) in the name by default otherwise we throw errors if they create a term with the same name at the same level
| setStagedName(properties.name); | ||
| form.setFieldValue('name', properties.name); | ||
| } |
There was a problem hiding this comment.
what do you think about doing ${properties.name} (copy) here? for both of these setters
There was a problem hiding this comment.
uupdated to this!
datahub-cyborg
bot
added
pending-submitter-merge
and removed
needs-review
github-actions
bot
requested a deployment
to
datahub-project-web-react (Preview)
This was missing from v2, but in v1; I dont think it ever made it over:
Before:
After:
Screen.Recording.2025-11-07.at.1.31.47.PM.mov